Skip to content

Comments

Create commit-check.yml#13

Closed
ynancher wants to merge 1 commit intoqualcomm-linux:mainfrom
ynancher:ynancher-kgsl
Closed

Create commit-check.yml#13
ynancher wants to merge 1 commit intoqualcomm-linux:mainfrom
ynancher:ynancher-kgsl

Conversation

@ynancher
Copy link

@ynancher ynancher commented Dec 9, 2025

Adds the commit-msg-check GitHub Action
Checks commit messages added in pull requests
Valid for any PRs raised on this repository
Enforces these checks:

Commit Subject: Must exist and be ≤ 50 characters
Commit Body: Each line must wrap at 72 characters
Blank Line Check: Ensures a blank line between subject and body Also checks for blank line before Signed-off-by

Adds the commit-msg-check GitHub Action
Checks commit messages added in pull requests
Valid for any PRs raised on this repository
Enforces these checks:

Commit Subject: Must exist and be ≤ 50 characters
Commit Body: Each line must wrap at 72 characters
Blank Line Check: Ensures a blank line between subject and body
Also checks for blank line before Signed-off-by

Signed-off-by: Yashasvi Nancherla <ynancher@qti.qualcomm.com>
@shivrawa
Copy link
Contributor

Hi @ynancher
These checks are ensured by the LBSA(Linux build service account) on the Gerrit.
we don't upload PR/code changes directly to the KGSL GitHub repo.

So i believe we don't need this change and we can close this PR. @ynancher Let me know your view also on this.

@ynancher
Copy link
Author

Hi @ynancher These checks are ensured by the LBSA(Linux build service account) on the Gerrit. we don't upload PR/code changes directly to the KGSL GitHub repo.

So i believe we don't need this change and we can close this PR. @ynancher Let me know your view also on this.

Sure @shivrawa , makes sense

@ynancher ynancher closed this Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants